Skip to content
/ contao-demo Public
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Replace the deprecated RSS reader with the Feed reader #60

Merged
merged 3 commits into from
Jan 31, 2025
Merged

Replace the deprecated RSS reader with the Feed reader #60

merged 3 commits into from
Jan 31, 2025

Conversation

de-es
Copy link
Contributor

@de-es de-es commented Jan 19, 2025

Depends on contao/contao#7904

@zoglo zoglo self-requested a review January 19, 2025 13:27
@zoglo zoglo added this to the 5.3 milestone Jan 19, 2025
@zoglo zoglo added the migration Migrating deprecated modules / etc to a new feature label Jan 19, 2025
@de-es de-es marked this pull request as ready for review January 20, 2025 20:40
zoglo
zoglo requested changes Jan 20, 2025
View reviewed changes
Copy link
Member

@zoglo zoglo left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did not test it yet but there's a few things to match the code style of the project :) (mostly nitpicks)

@zoglo
Copy link
Member

zoglo commented Jan 20, 2025

@de-es I've tested everything, and would be okay with it once the requested changes are in.
The biggest change here would be that it's called Feed Reader rather than RSS Reader but that's just a chore

The wrong order is a bug anyways: contao/contao#7905

@zoglo zoglo requested a review from sascha-mueller January 20, 2025 22:16
@de-es @zoglo
CS
a3d3cd3 
Co-authored-by: Sebastian Zoglowek <55794780+zoglo@users.noreply.github.com>
@zoglo zoglo changed the title Replace deprecated RSS reader with feed reader Replace the deprecated RSS reader with the Feed reader Jan 31, 2025
@zoglo zoglo merged commit b38f9a0 into contao:5.3.x Jan 31, 2025
@zoglo
Copy link
Member

zoglo commented Jan 31, 2025

Thank you @de-es

zoglo added a commit that referenced this pull request Jan 31, 2025
@zoglo
Match the module-feed-reader styles to the deprecated rss-feed
7561c33 
* see #60 for more details
@de-es de-es deleted the feed-reader branch January 31, 2025 21:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zoglo zoglo

@sascha-mueller sascha-mueller

Assignees

@de-es de-es

Labels
migration Migrating deprecated modules / etc to a new feature
Projects
None yet
Milestone
5.3
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@de-es @zoglo